Skip to content

Conversation

@butonic
Copy link
Member

@butonic butonic commented Sep 6, 2017

needed for and requires owncloud/core@3783534 because the migration manipulates the db filecache directly. bypassing the core implementation, which would take care of cleaning the cache

@butonic butonic added this to the development milestone Sep 6, 2017
@butonic butonic self-assigned this Sep 6, 2017
@butonic butonic requested a review from tomneedham September 6, 2017 07:54
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@PVince81
Copy link
Contributor

PVince81 commented Sep 6, 2017

why not put this small commit directly into owncloud/core#28166 ?

@butonic
Copy link
Member Author

butonic commented Sep 6, 2017

@PVince81 because it is a different app?

@PVince81
Copy link
Contributor

PVince81 commented Sep 6, 2017

🤦‍♂️ you're right... I forgot that encryption app is separate on master

@felixboehm
Copy link

@tomneedham please review. blocking QA / release

@tomneedham
Copy link
Contributor

This PR is only required if the core PR: owncloud/core#28166 is merged in - else it can be ignored

@tomneedham tomneedham modified the milestones: development, planned Oct 17, 2017
@butonic
Copy link
Member Author

butonic commented Feb 18, 2019

closing in favor of OCIS

@butonic butonic closed this Feb 18, 2019
@butonic butonic deleted the clear-cache-manually branch February 18, 2019 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants